Skip to content

fix(sync/l2): retry fetching signature until we succeed#3023

Merged
kkovaacs merged 2 commits intomainfrom
krisztian/retry-fetching-signature-until-success
Sep 25, 2025
Merged

fix(sync/l2): retry fetching signature until we succeed#3023
kkovaacs merged 2 commits intomainfrom
krisztian/retry-fetching-signature-until-success

Conversation

@kkovaacs
Copy link
Contributor

@kkovaacs kkovaacs commented Sep 24, 2025

There is a race condition with the current setup of Starknet feeder gateway: we've observed that fetching the signature sometimes fails even after fetching the block itself succeeds. We currently fail on such errors and exit the L2 sync task, which is not ideal.

This change modifies the logic to retry fetching the signature until it succeeds.

@kkovaacs kkovaacs marked this pull request as ready for review September 24, 2025 13:21
@kkovaacs kkovaacs requested a review from a team as a code owner September 24, 2025 13:21
@kkovaacs kkovaacs force-pushed the krisztian/retry-fetching-signature-until-success branch from e7f86a5 to ec06039 Compare September 25, 2025 08:35
There is a race condition with the current setup of Starknet feeder
gateway: we've observed that fetching the signature sometimes fails
even after fetching the block itself succeeds. We currently fail on such
errors and exit the L2 sync task, which is not ideal.

This change modifies the logic to retry fetching the signature until
it succeeds.
@kkovaacs kkovaacs force-pushed the krisztian/retry-fetching-signature-until-success branch from ec06039 to fa99523 Compare September 25, 2025 09:32
@kkovaacs kkovaacs merged commit 5684e99 into main Sep 25, 2025
8 checks passed
@kkovaacs kkovaacs deleted the krisztian/retry-fetching-signature-until-success branch September 25, 2025 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants